Skip to content

ZA2 - Ele Mashige#365

Open
EleMashige wants to merge 6 commits into
CodeYourFuture:masterfrom
EleMashige:master
Open

ZA2 - Ele Mashige#365
EleMashige wants to merge 6 commits into
CodeYourFuture:masterfrom
EleMashige:master

Conversation

@EleMashige
Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

Copy link
Copy Markdown

@Mathias02 Mathias02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El, would you please use semantic tags in your HTML and add your comments accordingly for a better visibility

Comment thread index.html
<li class="list-items"><a href="">Login</a></li>
</ul>
</nav>
<!-- BACKGROUND -->
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment seems to be not giving a meaningful sense here

Comment thread index.html
<img class="promo-img" src="/Users/admin/Documents/Ele clone/HTML-CSS-Module-Project/img/icon-refill.svg" alt="pay img">
<h3>Pay as You Go</h3>
</div>
<!-- SOCIALS -->
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment stands offside

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants